Skip to content

fix: incorrect hex representation of Unicode characters (Example: \x2019 instead of \x{2019}) #222

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Apr 17, 2025

Conversation

soujanyanmbri
Copy link
Contributor

Reference Issue: #221

Copy link
Member

@fzipi fzipi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your proposed fix is not what you implemented here, or I'm reading it wrong? 🤔

@fzipi fzipi changed the title Incorrect hex representation of Unicode characters (Example: \x2019 instead of \x{2019}) fix: incorrect hex representation of Unicode characters (Example: \x2019 instead of \x{2019}) Apr 15, 2025
@soujanyanmbri
Copy link
Contributor Author

Your proposed fix is not what you implemented here, or I'm reading it wrong? 🤔

Oops, missed the update in the issue. \u2019 does not work with go regex, but \x{2019} works everywhere. Updated the issue accordingly.
CC: @fzipi

@soujanyanmbri soujanyanmbri requested a review from fzipi April 15, 2025 12:29
@fzipi fzipi requested a review from theseion April 15, 2025 12:36
Copy link
Member

@fzipi fzipi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would wait for @theseion, but LGTM.

soujanyanmbri and others added 3 commits April 16, 2025 11:02
Co-authored-by: Max Leske <250711+theseion@users.noreply.github.com>
Co-authored-by: Max Leske <250711+theseion@users.noreply.github.com>
@soujanyanmbri soujanyanmbri requested a review from theseion April 16, 2025 05:46
Copy link
Contributor

@theseion theseion left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! Much appreciated.

@theseion
Copy link
Contributor

@soujanyanmbri please consider signing your commits. We have policies in place that require commits to be signed.

@theseion theseion merged commit 7dcb95b into coreruleset:main Apr 17, 2025
3 checks passed
@theseion theseion added the bug Something isn't working label Apr 17, 2025
@soujanyanmbri
Copy link
Contributor Author

@soujanyanmbri please consider signing your commits. We have policies in place that require commits to be signed.

Sure!! Will do it next time. Thank you :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants